Skip to content

Harden script bridge and workflow governance - #24

Closed
generalio wants to merge 7 commits into
mainfrom
fix/workflow-governance-hardening
Closed

Harden script bridge and workflow governance#24
generalio wants to merge 7 commits into
mainfrom
fix/workflow-governance-hardening

Conversation

@generalio

@generalio generalio commented Aug 18, 2026

Copy link
Copy Markdown
Owner

目的

修复全项目审查发现的 JS 桥来源校验、可信主机可变集合、跨平台状态测试与工作流治理问题,并使默认流程不再要求阶段交接文档。

改动内容

  • 收紧 Android、iOS、Desktop 与 extension 的脚本桥可信 HTTPS 来源、主框架和默认端口校验。
  • 为可信主机集合建立不可变快照,补齐桥接和 fixtures 回归测试。
  • 将 workflow 的阶段交接文档改为维护者显式要求时才创建;停止状态必须携带原因和唯一下一步。
  • 将默认 Agent 编排收敛为顺序的 Planner、单一 Implementer、Integrator、Verify-Reviewer;禁止按平台自动拆分,第二个 Implementer 必须经 Planner 书面批准并满足隔离约束。
  • 对齐账本 README、角色协议、任务模板与 CLI 行为,补齐文档回归测试。

平台影响

  • Android:受影响;验证证据:unit test、androidTest Kotlin 编译和 release assemble 通过。
  • iOS:受影响;验证证据:iosSimulatorArm64 测试 Kotlin 编译通过,未执行 WKWebView runtime。
  • Desktop:受影响;验证证据:Desktop JVM 测试通过。
  • JS/Wasm:受影响;验证证据:示例 JS/Wasm 编译通过;runtime 测试受 Yarn 网络互斥限制未执行。

验证命令与结果

  • PYTHONDONTWRITEBYTECODE=1 python3 -m unittest tools.tests.test_agent_workflow -v:48 项通过。
  • git diff origin/main...3ba4d6d41c327650ace3555aebc832cce1e71275 --check:通过。
  • ./gradlew apiCheck --console=plain --no-daemon --no-parallel --max-workers=1:通过。
  • extension、fixtures、Desktop JVM 测试,Android unit test/androidTest Kotlin 编译,iOS Kotlin 编译,以及 Android release 和示例 JS/Wasm 编译:低并发逐模块命令均通过。

风险与兼容性

  • 公共 API:不受影响;验证证据:apiCheck 通过且 API 基线未改。
  • JS 桥默认保持最小权限:仅精确 HTTPS 主机、443 默认端口和可信主框架可访问;未放宽 Cookie、文件访问、下载或外部导航权限。
  • 未执行 Android System WebView/WKWebView runtime;JS/Wasm runtime 测试受 Yarn 网络互斥限制未执行。Windows 或缺少 fcntl 的工作流 CLI 属于既有平台边界。

文档变更

更新 AGENTS、工作流协议、任务模板和账本 README,说明默认无 handoffs、停止参数与锁获取顺序、最小角色编排,以及完整审查/恢复证据规则。

Agent 工作流记录

  • 任务契约路径:.codex/workflow/runs/MW-20260818-004/task-contract.md
  • 候选提交 SHA:3ba4d6d41c327650ace3555aebc832cce1e71275
  • 账本验证:PASS
  • 审查裁决:PASS

Copilot AI lite review requested due to automatic review settings August 18, 2026 03:19

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@generalio generalio closed this Aug 18, 2026
@generalio
generalio deleted the fix/workflow-governance-hardening branch August 18, 2026 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants